Skip to content

Conversation

@mawi-li
Copy link
Member

@mawi-li mawi-li commented Aug 31, 2025

GitHub Actions Workflow Update – Windows Only Support

The download.yml workflow file has been successfully updated! Linux and macOS builds were removed, keeping only the Windows version. Additionally, permissions blocks were added to resolve GITHUB_TOKEN issues.

Key Changes:

  • Added permissions: contents: read at the global workflow level
  • Added permissions: contents: read for the first job download-info
  • Added permissions: {} for the second job test-downloads
  • Removed all Linux/macOS references from Display Download Information
  • Removed Linux/macOS installation commands
  • Removed Linux/macOS download test steps
  • Kept Windows-only build across all workflow sections

✅ Now download.yml supports Windows only and no longer triggers GITHUB_TOKEN permissions warnings.

🔄 Pull Request Overview

Affected Component

  • CIN CLI
  • CIN Framework Core
  • CIN Library
  • Documentation
  • Website
  • Other: ___________

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎨 Code style/formatting changes
  • ♻️ Code refactoring (no functional changes)
  • ⚡ Performance improvements
  • 🔒 Security improvements
  • 🧪 Tests
  • 🔧 Build/CI changes

📝 Description

What does this PR do?

Why is this change needed?

How has this been tested?

  • Unit tests
  • Integration tests
  • Manual testing
  • Browser testing (if applicable)
  • CLI testing (if applicable)

🔗 Related Issues

Closes #
Related to #

📸 Screenshots/Examples

Before:

After:

Checklist

Code Quality

  • My code follows the CIN Framework coding standards (PSR-12)
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings or errors
  • I have added proper PHPDoc blocks for all new functions/methods

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this change in multiple environments (if applicable)

Documentation

  • I have made corresponding changes to the documentation
  • I have updated the README.md if needed
  • I have updated the CHANGELOG.md if needed

Security & Performance

  • My changes do not introduce any security vulnerabilities
  • My changes do not negatively impact performance
  • I have followed CIN Framework security best practices

🚀 Deployment Notes

📋 Additional Context

🔍 Review Guidelines

  • Code follows CIN Framework standards
  • Tests are comprehensive and pass
  • Documentation is updated
  • Security considerations are addressed
  • Performance impact is acceptable

CIN FRAMEWORK – WEB 4 – POWERED BY LOCAL SELF-SUFFICIENCY AND AGGRESSIVE SECURITY

Thank you for contributing to CIN Framework! 🙏

## GitHub Actions Workflow Update – Windows Only Support

The `download.yml` workflow file has been successfully updated!
Linux and macOS builds were removed, keeping only the Windows version. Additionally, `permissions` blocks were added to resolve **GITHUB_TOKEN** issues.

### Key Changes:
- Added `permissions: contents: read` at the global workflow level
- Added `permissions: contents: read` for the first job **download-info**
- Added `permissions: {}` for the second job **test-downloads**
- Removed all Linux/macOS references from **Display Download Information**
- Removed Linux/macOS installation commands
- Removed Linux/macOS download test steps
- Kept **Windows-only** build across all workflow sections

✅ Now `download.yml` supports **Windows only** and no longer triggers **GITHUB_TOKEN permissions** warnings.
@mawi-li mawi-li requested a review from a team August 31, 2025 21:16
@mawi-li mawi-li self-assigned this Aug 31, 2025
@mawi-li mawi-li added the bug Something isn't working label Aug 31, 2025
@mawi-li mawi-li merged commit 920d76c into main Aug 31, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant